Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TACODEV-774: enable metric on argo series #108

Merged
merged 2 commits into from
Jul 8, 2021
Merged

Conversation

intelliguy
Copy link
Contributor

Objective
Argo worflow/CD도 prometheus metric으로 수집할 수 있다. Argo worklfow/CD의 metric을 조사하고, Argo CD 의 주요 metric 수집한다.
argo cd/workflow exporter 존재함, 서비스 모니터 생성해서 prometheus로 데이터 저장 확인 (lma-addons)
grafana dashboard 까지 추가, 확인 (grafana labs에 있는 대시보드 활용)

Doing with this PR
argo 설치시 metric을 생성할 수 있도록 설정

@intelliguy intelliguy requested review from zugwan and estherk0 July 1, 2021 08:55
@bluejayA bluejayA requested a review from Jaesang July 1, 2021 09:01
roles/decapod/tasks/argo.yml Show resolved Hide resolved
@robertchoi80
Copy link
Contributor

argo-workflow v3 로 가면서 사용하는 차트가 바뀌어서 value override하는 부분에서 문제가 있을것 같습니다ㅠ
#109

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 3 days.

@github-actions github-actions bot added the Stale label Jul 5, 2021
@estherk0 estherk0 requested a review from zugwan July 6, 2021 04:16
Comment on lines 29 to 30
metricsConfig:
enabled: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분 잘 못 들어간 것 같습니다. 밑에 있는 metricsConfig: 랑 겹치는데 삭제해도 되지 않을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네 맞네요...수정해서 올리겠습니다.

@intelliguy intelliguy removed the Stale label Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants